(byte-compile-file): Don't write output if error.

diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index 76a0712dcad5bcf23f31c101ad9d4dbae2959ed3..7c7a2f1c13bca10bf2b59f6881f1b3421d35c1a3 100644 (file)
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -1159,41 +1159,41 @@ With prefix arg (noninteractively: 2nd arg), load the file after compiling."
     ;; within byte-compile-from-buffer lingers in that buffer.
     (setq output-buffer (byte-compile-from-buffer input-buffer))
     (or byte-compiler-error-flag
-       (kill-buffer input-buffer))
-    (save-excursion
-      (set-buffer output-buffer)
-      (goto-char (point-max))
-      (insert "\n")                    ; aaah, unix.
-      (let ((vms-stmlf-recfm t))
-       (setq target-file (byte-compile-dest-file filename))
-;;     (or byte-compile-overwrite-file
-;;         (condition-case ()
-;;             (delete-file target-file)
-;;           (error nil)))
-       (if (file-writable-p target-file)
-           (let ((kanji-flag nil))     ; for nemacs, from Nakagawa Takayuki
-             (write-region 1 (point-max) target-file))
-         ;; This is just to give a better error message than
-         ;; write-region
-         (signal 'file-error
-                 (list "Opening output file"
-                       (if (file-exists-p target-file)
-                           "cannot overwrite file"
-                         "directory not writable or nonexistent")
-                       target-file)))
-;;     (or byte-compile-overwrite-file
-;;         (condition-case ()
-;;             (set-file-modes target-file (file-modes filename))
-;;           (error nil)))
-       )
-      (kill-buffer (current-buffer)))
-    (if (and byte-compile-generate-call-tree
-            (or (eq t byte-compile-generate-call-tree)
-                (y-or-n-p (format "Report call tree for %s? " filename))))
-        (save-excursion
-         (display-call-tree filename)))
-    (if load
-       (load target-file)))
+       (kill-buffer input-buffer)
+       (save-excursion
+         (set-buffer output-buffer)
+         (goto-char (point-max))
+         (insert "\n")                 ; aaah, unix.
+         (let ((vms-stmlf-recfm t))
+           (setq target-file (byte-compile-dest-file filename))
+    ;; (or byte-compile-overwrite-file
+    ;;     (condition-case ()
+    ;;         (delete-file target-file)
+    ;;       (error nil)))
+           (if (file-writable-p target-file)
+               (let ((kanji-flag nil)) ; for nemacs, from Nakagawa Takayuki
+                 (write-region 1 (point-max) target-file))
+             ;; This is just to give a better error message than
+             ;; write-region
+             (signal 'file-error
+                     (list "Opening output file"
+                           (if (file-exists-p target-file)
+                               "cannot overwrite file"
+                             "directory not writable or nonexistent")
+                           target-file)))
+    ;; (or byte-compile-overwrite-file
+    ;;     (condition-case ()
+    ;;         (set-file-modes target-file (file-modes filename))
+    ;;       (error nil)))
+           )
+         (kill-buffer (current-buffer)))
+       (if (and byte-compile-generate-call-tree
+                (or (eq t byte-compile-generate-call-tree)
+                    (y-or-n-p (format "Report call tree for %s? " filename))))
+           (save-excursion
+             (display-call-tree filename)))
+       (if load
+           (load target-file))))
   t)
 
 ;;(defun byte-compile-and-load-file (&optional filename)